Skip to content

Conversation

wecharyu
Copy link
Contributor

What changes were proposed in this pull request?

Introduce a new SQL to enable unset SERDE properties:

ALTER TABLE table_identifier partition_spec UNSET SERDEPROPERTIES [ IF EXISTS ] ( key1, key2, ... )

Why are the changes needed?

Spark SQL already supports SET SERDEPROPERTIES, it would be more convenient if we can use UNSET SERDEPROPETIES

Does this PR introduce any user-facing change?

Yes, user can use this new SQL.

How was this patch tested?

Add unit tests:

./build/mvn  test -Dtest=none -Dsuites="org.apache.spark.sql.execution.command.AlterTableUnsetSerdePropertiesParserSuite" -pl :spark-sql_2.13
./build/mvn  test -Dtest=none -Dsuites="org.apache.spark.sql.execution.command.v1.AlterTableUnsetSerdePropertiesSuite" -pl :spark-sql_2.13
./build/mvn  test -Dtest=none -Dsuites="org.apache.spark.sql.hive.execution.command.AlterTableUnsetSerdePropertiesSuite" -pl :spark-hive_2.13
./build/mvn  test -Dtest=none -Dsuites="org.apache.spark.sql.execution.command.v2.AlterTableUnsetSerdePropertiesSuite" -pl :spark-sql_2.13

Was this patch authored or co-authored using generative AI tooling?

No.

Copy link

github-actions bot commented Sep 4, 2025

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Sep 4, 2025
@github-actions github-actions bot closed this Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant